Skip to content

update nvlink checks#143

Open
ncclementi wants to merge 1 commit intorapidsai:mainfrom
ncclementi:update-nvlink-check
Open

update nvlink checks#143
ncclementi wants to merge 1 commit intorapidsai:mainfrom
ncclementi:update-nvlink-check

Conversation

@ncclementi
Copy link
Copy Markdown

Closes #142

This PR (mostly made with Claude):

  • makes sure that we don't raise when NVLink it's not supported,
  • makes sure that we don't only check NVLink for device 0.
  • Has better raises, to give more informative information to the user when the check fails.
  • Updated the tests to make sure we check all the different scenarios.

@ncclementi ncclementi requested a review from a team as a code owner March 30, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nvlink check output not very helpful

1 participant